Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Handle failing case for update account cache in require #9989

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

sorpaas
Copy link
Collaborator

@sorpaas sorpaas commented Nov 29, 2018

Fixes #9838

Handle the case when updating account cache fails, but we're inside RefMut.

@sorpaas sorpaas added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Nov 29, 2018
@5chdn 5chdn added this to the 2.3 milestone Nov 29, 2018
@ascjones ascjones added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Dec 3, 2018
@sorpaas sorpaas merged commit 60691d0 into master Dec 3, 2018
@5chdn 5chdn deleted the sp-require-account-cache branch December 4, 2018 11:29
@5chdn 5chdn mentioned this pull request Dec 5, 2018
3 tasks
5chdn added a commit that referenced this pull request Dec 5, 2018
* version: bump beta to 2.2.4

* ci: move future releases to ethereum subdir on s3 (#10017)

* ci: move future releases to ethereum subdir on s3

* ci: redesign s3 bucket logic

* ci: use the releases bucket

* Handle failing case for update account cache in require (#9989)

* state: remove accidently merged test

* state: remove merge leftovers
insipx pushed a commit to insipx/parity-ethereum that referenced this pull request Dec 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants